Skip to content

Don't try to mock UnitOfWork as it's final#2843

Merged
dunglas merged 1 commit into
api-platform:2.4from
teohhanhui:fix/mongodb-odm-uow-final
Jun 7, 2019
Merged

Don't try to mock UnitOfWork as it's final#2843
dunglas merged 1 commit into
api-platform:2.4from
teohhanhui:fix/mongodb-odm-uow-final

Conversation

@teohhanhui

@teohhanhui teohhanhui commented Jun 5, 2019

Copy link
Copy Markdown
Contributor
Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets N/A
License MIT
Doc PR N/A

Also re-add mongodb packages to require-dev as we want developers to be able to run phpstan successfully. This means they must have ext-mongodb, but it's easy to satisfy this requirement...

Comment thread .circleci/config.yml
Comment thread src/Test/DoctrineMongoDbOdmTestCase.php Outdated
@teohhanhui
teohhanhui force-pushed the fix/mongodb-odm-uow-final branch 3 times, most recently from 99c6c43 to 5928b4f Compare June 6, 2019 11:13
@teohhanhui

Copy link
Copy Markdown
Contributor Author

Need to fix this: doctrine/DoctrineMongoDBBundle#557

@teohhanhui
teohhanhui force-pushed the fix/mongodb-odm-uow-final branch 2 times, most recently from 09c98a2 to 828ba3e Compare June 6, 2019 12:48
Comment thread composer.json
"behat/symfony2-extension": "^2.1.1",
"behatch/contexts": "3.1.0",
"doctrine/annotations": "^1.2",
"doctrine/data-fixtures": "^1.2.2",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Temporary workaround for doctrine/DoctrineMongoDBBundle#557

@teohhanhui
teohhanhui force-pushed the fix/mongodb-odm-uow-final branch from 828ba3e to 2e0dced Compare June 6, 2019 13:09
@dunglas
dunglas merged commit d33b6d2 into api-platform:2.4 Jun 7, 2019
@dunglas

dunglas commented Jun 7, 2019

Copy link
Copy Markdown
Member

Thanks @teohhanhui!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants